@ogazboiz Thanks for doing this work. I do appreciate it.

Look at this comment: #57 (comment)

It says this:

At this point interfaces may be made and put in an interfaces directory, so they can be accessed by users in the future. Only standard interfaces such as ERC20, ERC721 etc. should be made. Interfaces may not be imported or used by contracts or libraries within Compose. That may change in the future.

Please remove any use of interfaces and put all the interfaces into the directory here: src/interfaces.

@ogazboiz Thanks for doing this work. I do appreciate it.

Look at this comment: #57 (comment)

It says this:

At this point interfaces may be made and put in an interfaces directory, so they can be accessed by users in the future. Only standard interfaces such as ERC20, ERC721 etc. should be made. Interfaces may not be imported or used by contracts or libraries within Compose. That may change in the future.

Please remove any use of interfaces and put all the interfaces into the directory here: src/interfaces.

alright i am doing that

I've implemented your feedback. All interface files are now in src/interfaces/ directory and I've removed all interface imports and usage from contracts and libraries. The interfaces are now standalone files for external users only, while all internal code uses local error definitions. @mudgen

@ogazboiz Good job. Merged.

i am really happy to contrbute

Hi, I see the contributor shows as "ogazboizz" but that should be "ogazboiz" (my personal account that pushed the code). Could you correct the contributor attribution? The code was developed and pushed from my personal GitHub account. @mudgen

can you help me with that?

Yes, sorry about that. Do you mean in the comment I left? Or somewhere else?

Yes, sorry about that. Do you mean in the comment I left? Or somewhere else?

Hi @mudgen!

I'm looking at the contributor section and it shows "ogazboizz" as the contributor, but that's not my account. It should show "ogazboiz" (my personal GitHub account that I used to develop and push the code).

Could you update the contributor attribution? Thanks for letting me contribute to this awesome project!

@ogazboiz Yes, I see. This is a weird thing that Github used the wrong username. Right now I don't see how to change this. Do you know how to change this?

@ogazboiz See this: https://docs.github.com/en/pull-requests/committing-changes-to-your-project/troubleshooting-commits/why-are-my-commits-linked-to-the-wrong-user

@ogazboiz To fix this I think you will need to make your email address associated with your @ogazboiz account rather than your @ogazboizz account, or give me another email address that I can change the commits to use which is associated with your @ogazboiz account.

Hi @mudgen!

Thanks for offering to help fix this! My personal email associated with my @ogazboiz account is: ogazboizakpolo@gmail.com

I've also created a follow-up PR #86 that addresses the attribution issue. The new PR adds author attribution to all interface files and will show the correct @ogazboiz account since I've fixed my Git configuration.

You can either use the email to update the existing commits, or check out PR #86 for the proper attribution going forward.

@ogazboiz
Okay, now that you have fixed your git configuration, I think the safe thing to do is this:

1. I can remove your changes from the repository.
2. You submit a new pull request with the same changes.
3. I merge those changes.

This way your correct account should show up in the contributors section and show what you have done.
Should we proceed with this plan?

alright @mudgen or you can just merge my follow up pr

alright @mudgen or you can just merge my follow up pr

No, I won't do that. We are not currently attributing authors like that, and I want the code to remain consistent. If contributors want better attribution or credit then we will figure out a consistent way to do that.

alright @mudgen or you can just merge my follow up pr

No, I won't do that. We are not currently attributing authors like that, and I want the code to remain consistent. If contributors want better attribution or credit then we will figure out a consistent way to do that.

no problem sir.
just let me know when you remove the changes so i can just create pr again

It is done. Ready for your new PR.

It is done. Ready for your new PR.

i have created a pr #89